Skip to content

refactor: update styling and improve component structure#11

Merged
awsl233777 merged 2 commits intoawsl-project:mainfrom
whhjdi:fix/theme
Jan 14, 2026
Merged

refactor: update styling and improve component structure#11
awsl233777 merged 2 commits intoawsl-project:mainfrom
whhjdi:fix/theme

Conversation

@whhjdi
Copy link
Contributor

@whhjdi whhjdi commented Jan 13, 2026

  • Adjusted spacing in ClientTypeRoutesContent for better layout.
  • Enhanced compatibility in useCooldowns hook by handling multiple field names.
  • Refactored ProviderRowContent to use Button component and improved visual feedback.
  • Updated OverviewPage to use linear gradient for consistency.
  • Refined ProviderCreateFlow for better readability and state management.
  • Improved SelectTypeStep with consistent styling and layout adjustments.

…us files

- Adjusted spacing in ClientTypeRoutesContent for better layout.
- Enhanced compatibility in useCooldowns hook by handling multiple field names.
- Refactored ProviderRowContent to use Button component and improved visual feedback.
- Updated OverviewPage to use linear gradient for consistency.
- Refined ProviderCreateFlow for better readability and state management.
- Improved SelectTypeStep with consistent styling and layout adjustments.
@Bowl42
Copy link
Collaborator

Bowl42 commented Jan 13, 2026

/review

@github-actions
Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 4 🔵🔵🔵🔵⚪
🧪 PR contains tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Styling Consistency

The new styling for the timer section should be reviewed for consistency with the overall design system, especially regarding color gradients and borders.

<div className="col-span-2 relative overflow-hidden rounded-xl bg-linear-to-br from-cyan-950/30 to-transparent border border-cyan-500/20 p-5 flex flex-col items-center justify-center group">
  <div className="absolute inset-0 bg-cyan-400/5 opacity-50 group-hover:opacity-100 transition-opacity" />
  <div className="relative flex items-center gap-1.5 text-cyan-500 mb-1">
Possible Issue

The backward compatibility handling for untilTime and until should be tested to ensure it works as expected across all scenarios.

const untilTime = cooldown.untilTime || (cooldown as Record<string, unknown>).until as string;
if (!untilTime) return 0;
Validation Logic

The validation logic in isValid should be thoroughly tested to ensure it correctly identifies valid and invalid provider configurations.

const isValid = () => {
  if (!formData.name.trim()) return false
  if (!formData.apiKey.trim()) return false
  const hasEnabledClient = formData.clients.some(c => c.enabled)
  const hasUrl =
    formData.baseURL.trim() ||
    formData.clients.some(c => c.enabled && c.urlOverride.trim())
  return hasEnabledClient && hasUrl
}

@awsl233777 awsl233777 merged commit 0726933 into awsl-project:main Jan 14, 2026
@whhjdi whhjdi deleted the fix/theme branch January 14, 2026 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants